-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Teddy.gesbert/doc dora #29393
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Teddy.gesbert/doc dora #29393
Conversation
Co-authored-by: Jorge Calvar <[email protected]>
Change deployment.merge_time to Merge Time (i.e. metric to field) to avoid any confusion with previous Metrics type
…tion into teddy.gesbert/doc-dora
Applied suggestions from Joe Peeples
Co-authored-by: Alicia Scott <[email protected]>
Co-authored-by: Alicia Scott <[email protected]>
- Added consistency between data sources requirements. - Remove /dora_metrics/failures/datadog_incidents alias as not needed.
Co-authored-by: Alicia Scott <[email protected]>
Added backfill documentation for retrieving past incidents using Incident Management as a DORA failure source.
Co-authored-by: May Lee <[email protected]>
Co-authored-by: May Lee <[email protected]>
Co-authored-by: May Lee <[email protected]>
Removing Preview alert + 2 small changes on definitions
Preview links (active after the
|
content/en/dora_metrics/_index.md
Outdated
@@ -29,7 +29,6 @@ further_reading: | |||
<div class="alert alert-warning">DORA Metrics is not available in the selected site ({{< region-param key="dd_site_name" >}}) at this time.</div> | |||
{{< /site-region >}} | |||
|
|||
<div class="alert alert-warning">DORA Metrics is in Preview.</div> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🥳
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me!
/merge |
View all feedbacks in Devflow UI.
The expected merge time in
|
* Replaced incidents by failures * v1 * v1.1 * Little update of some requirements (DORA settings) * Upper case small changes * Update content/en/dora_metrics/data_collected/_index.md Co-authored-by: Jorge Calvar <[email protected]> * Applied feedback from Jorge Change deployment.merge_time to Merge Time (i.e. metric to field) to avoid any confusion with previous Metrics type * Applied feedback from Guillaume * Fix typos and improve consistency in DORA metrics documentation Applied suggestions from Joe Peeples * Applying Joe's feedback and fixing small tweak * Added Incident Management as a DORA Failure data source * Update content/en/dora_metrics/setup/failures.md Co-authored-by: Alicia Scott <[email protected]> * Update content/en/dora_metrics/setup/failures.md Co-authored-by: Alicia Scott <[email protected]> * Applied feedback from alicia - Added consistency between data sources requirements. - Remove /dora_metrics/failures/datadog_incidents alias as not needed. * Applied feedback from Jorge * Update content/en/dora_metrics/setup/failures.md Co-authored-by: Alicia Scott <[email protected]> * Update failures.md * Added backfill documentation Added backfill documentation for retrieving past incidents using Incident Management as a DORA failure source. * Update content/en/dora_metrics/setup/failures.md Co-authored-by: May Lee <[email protected]> * Update content/en/dora_metrics/setup/failures.md Co-authored-by: May Lee <[email protected]> * Update content/en/dora_metrics/setup/failures.md Co-authored-by: May Lee <[email protected]> * Added, TTR calculation + Pro tips * Added more details on TTR calculation * Removing Preview alert Removing Preview alert + 2 small changes on definitions * Update documentation to add rebase logic --------- Co-authored-by: Jorge Calvar <[email protected]> Co-authored-by: Alicia Scott <[email protected]> Co-authored-by: May Lee <[email protected]>
What does this PR do? What is the motivation?
Updated the DORA Metrics limitations to better describe how do we handle rebasing.
Merge instructions
Merge readiness:
If your branch doesn't follow this format, rename it or create a new branch and PR.
To have your PR automatically merged after it receives the required reviews, add the following PR comment:
Additional notes